deps: bump eslint from 9.39.2 to 10.0.0 in the development group - #3
Merged
diegosouzapw merged 1 commit intoFeb 13, 2026
Merged
Conversation
Bumps the development group with 1 update: [eslint](https://github.com/eslint/eslint). Updates `eslint` from 9.39.2 to 10.0.0 - [Release notes](https://github.com/eslint/eslint/releases) - [Commits](eslint/eslint@v9.39.2...v10.0.0) --- updated-dependencies: - dependency-name: eslint dependency-version: 10.0.0 dependency-type: direct:development update-type: version-update:semver-major dependency-group: development ... Signed-off-by: dependabot[bot] <support@github.com>
Contributor
|
@codex review |
|
To use Codex here, create a Codex account and connect to github. |
dependabot
Bot
deleted the
dependabot/npm_and_yarn/development-55a9f41e4a
branch
February 13, 2026 19:52
diegosouzapw
added a commit
that referenced
this pull request
Feb 28, 2026
…ance, robustness ## Critical Fixes - #1: Server readiness — waitForServer() polls before loading window - #2: Restart timeout — 5s + SIGKILL prevents IPC handler from hanging - #3: changePort — now stops/restarts server on new port ## Important Fixes - #4: Tray cleanup — destroy old Tray before recreating - #5: IPC emission — server-status & port-changed events - #6: Disposer pattern — replaces removeAllListeners - #7: useSyncExternalStore — eliminates 5x re-renders ## Minor: #8-#16 (dead code, CSP, platform titlebar, types, errors, version) Tests: 76 / 15 suites (was 64/9)
diegosouzapw
added a commit
that referenced
this pull request
Mar 7, 2026
…ance, robustness ## Critical Fixes - #1: Server readiness — waitForServer() polls before loading window - #2: Restart timeout — 5s + SIGKILL prevents IPC handler from hanging - #3: changePort — now stops/restarts server on new port ## Important Fixes - #4: Tray cleanup — destroy old Tray before recreating - #5: IPC emission — server-status & port-changed events - #6: Disposer pattern — replaces removeAllListeners - #7: useSyncExternalStore — eliminates 5x re-renders ## Minor: #8-#16 (dead code, CSP, platform titlebar, types, errors, version) Tests: 76 / 15 suites (was 64/9)
diegosouzapw
added a commit
that referenced
this pull request
Mar 14, 2026
Thanks @kfiramar! 🎉 Critical security fix — different startup paths were generating different `STORAGE_ENCRYPTION_KEY` values over the same SQLite database, causing `Unsupported state or unable to authenticate data` for all stored tokens. Improvements added on top: - Normalized `overridePath?.trim()` in `electron/main.js` to match `bootstrap-env.mjs` (addresses kilo-code-bot warning #1) - Added explanatory comment documenting the `preferredEnv` merge order intent in Electron startup (addresses kilo-code-bot warning #3) 4 commits + 113-line test file. The fail-closed behaviour (refusing to mint a new key when encrypted rows exist) is an excellent safeguard. Merged!
6 tasks
diegosouzapw
pushed a commit
that referenced
this pull request
Mar 23, 2026
- FIX #1: Add null check for cred.password (prevent undefined access) - FIX #2: Prioritize actual credentials over hardcoded account patterns - FIX #3: Convert CommonJS require() to ES imports for consistency - FIX #4: Move to App Router, add credential metadata response, document maintainer integration Additional improvements: - Better TypeScript error typing with optional chaining - Improved error messages for missing dependencies - Added maintainer TODO for provider system integration - Proper Next.js App Router format (route.ts) All bot warnings resolved. Ready for maintainer review.
prakersh
pushed a commit
to prakersh/OmniRoute
that referenced
this pull request
Mar 26, 2026
…-loading Thanks @kfiramar! 🎉 Critical security fix — different startup paths were generating different `STORAGE_ENCRYPTION_KEY` values over the same SQLite database, causing `Unsupported state or unable to authenticate data` for all stored tokens. Improvements added on top: - Normalized `overridePath?.trim()` in `electron/main.js` to match `bootstrap-env.mjs` (addresses kilo-code-bot warning diegosouzapw#1) - Added explanatory comment documenting the `preferredEnv` merge order intent in Electron startup (addresses kilo-code-bot warning diegosouzapw#3) 4 commits + 113-line test file. The fail-closed behaviour (refusing to mint a new key when encrypted rows exist) is an excellent safeguard. Merged!
prakersh
pushed a commit
to prakersh/OmniRoute
that referenced
this pull request
Mar 26, 2026
- FIX diegosouzapw#1: Add null check for cred.password (prevent undefined access) - FIX diegosouzapw#2: Prioritize actual credentials over hardcoded account patterns - FIX diegosouzapw#3: Convert CommonJS require() to ES imports for consistency - FIX diegosouzapw#4: Move to App Router, add credential metadata response, document maintainer integration Additional improvements: - Better TypeScript error typing with optional chaining - Improved error messages for missing dependencies - Added maintainer TODO for provider system integration - Proper Next.js App Router format (route.ts) All bot warnings resolved. Ready for maintainer review.
22 tasks done
4 of 12 tasks
diegosouzapw
pushed a commit
that referenced
this pull request
Apr 3, 2026
Fly.io Launch config files
payne0420
pushed a commit
to payne0420/OmniRoute
that referenced
this pull request
Apr 25, 2026
Round of fixes addressing the gemini-code-assist and chatgpt-codex review comments on the initial PR. ## High priority - **PoW solver no longer blocks the event loop** (gemini #1, #2). The 100k prekey solver and 500k proof-of-work solver were synchronous SHA3-512 loops that pinned a CPU core for tens to hundreds of milliseconds per request. Both are now async and `await`-yield to the event loop every 1000 iterations via setImmediate, so concurrent requests and I/O still get scheduled. Wall time is approximately the same; what changes is fairness, not throughput. - **Real upstream streaming for stream=true requests** (codex diegosouzapw#6). The conv call now passes `stream: true` through to the TLS client when the caller asked for streaming. The TLS client uses tls-client-node's streamOutputPath primitive to write the response body to a temp file as it arrives, and we tail that file as a ReadableStream so clients see chunks in real time instead of getting one buffered burst at the end. Also peeks the first 256 bytes — if the response starts with `{` it's almost certainly a JSON error envelope, so we wait for the full body and surface as a non-streaming error response. ## Medium priority - **Per-cookie device id** (gemini diegosouzapw#3). Replaced the single process-wide DEVICE_ID with a per-cookie SHA-256-derived UUID that's stable across requests for one connection but unique per cookie. This matches how the browser's persistent oai-did cookie behaves and avoids cross-account fingerprint sharing. Cache is bounded to 200 entries with FIFO eviction. - **Removed dead conv-cache code** (gemini diegosouzapw#4). The convCache / convLookup / convStore trio (~70 LOC) was unused — conversationId is hard-pinned to null because Temporary Chat conversation_ids 404 on reuse. Deleted entirely; the comment explains why we don't persist. - **No more console.log in the conv 4xx path** (gemini diegosouzapw#5). Replaced with log?.warn so it respects the application's logging configuration. - **Bound the warmup cache** (codex diegosouzapw#7). The (cookie, accessToken) -> timestamp map was unbounded; long-running multi-user deployments with rotating tokens would grow it forever. Now capped at 200 entries with FIFO eviction (Map iteration order = insertion order). - **Honor abort signals in TLS fetch** (codex diegosouzapw#8). tlsFetchChatGpt now checks options.signal before issuing the upstream call, after the call returns, and the streaming body listens for abort to stop tailing the temp file. tls-client-node's koffi binding can't cancel an in-flight request mid-call, but we no longer process / re-emit a response that the caller has already given up on. ## Tests All 27 chatgpt-web tests still pass; updated several to find calls by URL via findIndex rather than hardcoded indices, since the warmup sequence (/me, /conversations, /models) and two-stage Sentinel (prepare + chat-requirements) shifted positional offsets. Manually verified end-to-end: - Non-streaming completions - Streaming completions (real-time chunks; SSE [DONE] terminator) - Multi-turn with full history each turn (memory preserved correctly) Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
11 tasks done
payne0420
pushed a commit
to payne0420/OmniRoute
that referenced
this pull request
Apr 25, 2026
- Drop signed-URL fallback from image resolver (memory principle diegosouzapw#3); a failed bytes-fetch now returns null so no broken markdown / leaked estuary URL reaches the client. - Clamp /v1/images/generations n at 4 with a 400 before any upstream call so a stray n=1000 can't pin the worker for hours. - Log a warning when the PoW solver exhausts iterations and falls back to an unsolved token; production can now see Sentinel difficulty changes. - Track total resident bytes in the chatgpt image cache and evict oldest until both the count and byte budgets fit. Default 256 MB, tunable via OMNIROUTE_CGPT_WEB_IMAGE_CACHE_MAX_MB. - Retry the celsius WebSocket once on transport error before any frame, sharing the deadline so the total wait never exceeds budget. - Surface OMNIROUTE_CGPT_WEB_IMAGE_TIMEOUT_MS for the async-image wait (default 180000); deployments can stretch during upstream queue-deep windows. - Minor: drop unnecessary Uint8Array copy in image route, narrow publicBaseUrlHeaders to host-shaped headers, consolidate PoW solvers into shared solvePow helper, remove dead lastIndex resets, align tokenCache eviction with sibling caches. Tests: 4 new (signed-URL drop, byte-cap eviction, byte-counter decrement on TTL evict, n>4 rejection). 61/61 chatgpt-web tests pass; typecheck + lint clean. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This was referenced May 28, 2026
This was referenced Aug 21, 2026
3 tasks done
This was referenced Sep 15, 2026
Closed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Bumps the development group with 1 update: eslint.
Updates
eslintfrom 9.39.2 to 10.0.0Release notes
Sourced from eslint's releases.
... (truncated)
Commits
4e6c4ac10.0.0ddd8a22Build: changelog update for 10.0.0bff9091feat: handleArray.fromAsyncinarray-callback-return(#20457)1ece282chore: ignore/docs/v9.xin link checker (#20452)034e139ci: add type integration test for@html-eslint/eslint-plugin(#20345)f3fbc2fchore: set@eslint/jsversion to 10.0.0 to skip releasing it (#20466)e978ddadocs: Update README4cecf83docs: Update READMEc79f0abdocs: Update READMEafc0681chore: remove scopeManager.addGlobals patch for typescript-eslint parser (#20...Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting
@dependabot rebase.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebasewill rebase this PR@dependabot recreatewill recreate this PR, overwriting any edits that have been made to it@dependabot show <dependency name> ignore conditionswill show all of the ignore conditions of the specified dependency@dependabot ignore <dependency name> major versionwill close this group update PR and stop Dependabot creating any more for the specific dependency's major version (unless you unignore this specific dependency's major version or upgrade to it yourself)@dependabot ignore <dependency name> minor versionwill close this group update PR and stop Dependabot creating any more for the specific dependency's minor version (unless you unignore this specific dependency's minor version or upgrade to it yourself)@dependabot ignore <dependency name>will close this group update PR and stop Dependabot creating any more for the specific dependency (unless you unignore this specific dependency or upgrade to it yourself)@dependabot unignore <dependency name>will remove all of the ignore conditions of the specified dependency@dependabot unignore <dependency name> <ignore condition>will remove the ignore condition of the specified dependency and ignore conditions